feat(console): Agent console reached by drill-down from a roster row (slice 3/6) - #86
Merged
Merged
Conversation
…(slice 3/6) Slice 3 of the console-navigation-ia ADR (studio#83, mockup §7.4). Relocates the existing Agent console (files/config + per-agent chat, agent-consoles.md Part C) so it's reached by drill-down from Fleet detail's roster instead of the always-visible flat `agents-wrap` selector. `agentConsole.ts` is untouched — its selector (`#agent-list`) still opens a console on a `[data-agent]` click; that element just moves inside `#fleet-detail` and is hidden via CSS (no longer the entry point users see). A roster row's name is now a button (`rowHtml`, carrying both the ECS service name and the short name as candidate lookup keys) that main.ts resolves against the hidden selector and dispatches a synthetic click on — reusing the exact, already-tested open/dial/teardown path rather than duplicating it. A `MutationObserver` on `#agent-console`'s own `hidden` attribute swaps the roster for the open console (Part A: drill-down, not tab-peer surfaces) without adding a second source of truth for what's open. Leaving the fleet (back to Fleets, or switching fleets) also closes any open console via its own wired Close button. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Merged
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Slice 3 of the console-navigation-ia ADR (studio#83, mockup §7.4). Builds on slice 2 (#85).
Summary
agents-wrapselector.agentConsole.tsis not modified. Its selector (#agent-list) still opens a console on a[data-agent]click; that element just moves inside#fleet-detailand is hidden via CSS — it's no longer the entry point users see, but stays wired exactly as before.rowHtml, carrying the ECS service name + short name as candidate lookup keys). Clicking it resolves against the hidden selector and dispatches a synthetic click on the matching[data-agent]button — reusing the existing, already-tested open/dial/teardown path rather than duplicating it.MutationObserveron#agent-console's ownhiddenattribute swaps the roster for the open console (Part A: drill-down, not tab-peer surfaces) — no second source of truth for "what's open."← Fleets, or switching fleets) also closes any open console, via its own already-wired Close button.agents.tomlentry logs an info note instead of silently doing nothing.Test plan
tsc --noEmitvitest run(96/96 passing, 2 new cases for the row-open affordance)vite build🤖 Generated with Claude Code